STM32WBA: Implement BLE support #66181
Merged
Conversation
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
erwango
requested review from
alwa-nordic,
jhedberg,
Vudentz,
ABOSTM and
FRASTM
as code owners
zephyrbot
added
area: Bluetooth Host
platform: STM32
zephyrbot
requested review from
asbjornsabo,
Desvauxm-st,
gautierg-st,
GeorgeCGV,
hermabe,
jori-nordic,
kruithofa,
sjanc,
Thalley and
theob-pro
|
The following west manifest projects have been modified in this Pull Request:
Note: This message is automatically posted and updated by the Manifest GitHub Action. |
erwango
added
TSC
zephyrbot
added
manifest-hal_stm32
DNM
|
Addind 'TSC' label to review the binary blob addition. Don't hesitate to remove if this process is not required anymore. |
jori-nordic
previously approved these changes
Dec 6, 2023
erwango
force-pushed
the
dew_wba_ble_up
branch
from
5332ac6 to
3099d4d
Compare
|
@jhedberg Are you fine with the changes ? |
Looks generally fine. One small comment (see inline) - you can probably fix that by the time that the module PR has been merged and you update west.yml here in this PR. |
Add HCI Driver for STM32WBA devices. Based on B91 HCI driver. Signed-off-by: Erwan Gouriou <[email protected]>
RAMCFG will be required for STM32WBA BLE support. Signed-off-by: Erwan Gouriou <[email protected]>
In order to enable BLE support on STM32WBA, following APIs are implemented: - HostStack_: BLE Controller scheduling - ll_sys_: Link layer API required for scheduling - UTIL_TIMER_: BLE Controller timer utility - LINKLAYER_PLAT_: BLE controller utilities Signed-off-by: Erwan Gouriou <[email protected]>
SRAM6 is used by RF and should be defined as RAM_NOCACHE to allow unaligned access reads. "IO" might be a better match but is not available on this arch. Signed-off-by: Erwan Gouriou <[email protected]>
We might have to do this differently: Configure rng default clock in .dtsi Set board specific config in .dts Signed-off-by: Erwan Gouriou <[email protected]>
To be compatible with BLE operation and BLE Controler configuration, update board clock configuration to work using a fixed core clock at 16MHz. Signed-off-by: Erwan Gouriou <[email protected]>
erwango
force-pushed
the
dew_wba_ble_up
branch
from
3099d4d to
56111f4
Compare
Update west manifest to point to hal_stm32 PR containing changes which allow to build a zephyr bluetooth application on STM32WBA. Prior building such application, user should run 'west blobs fetch stm32' to install binary blobs providing the BLE controller support. Signed-off-by: Erwan Gouriou <[email protected]>
erwango
force-pushed
the
dew_wba_ble_up
branch
from
56111f4 to
4351188
Compare
jhedberg
approved these changes
Dec 18, 2023
FRASTM
approved these changes
Dec 18, 2023
jori-nordic
approved these changes
Dec 18, 2023
|
@carlescufi @nashif do you know if the vote for the blob happened? |
yes |
fabiobaltieri
merged commit 44af61b
into
zephyrproject-rtos:main
24 of 27 checks passed
mathieuchopstm
added a commit
to mathieuchopstm/zephyr
that referenced
this pull request
Jul 31, 2024
Remove the duplicate `USE_STM32_HAL_RAMCFG` Kconfig symbol definition from Kconfig.stm32, which was introduced by accident in PR zephyrproject-rtos#66181, commit 6ed002d. Signed-off-by: Mathieu Choplain <[email protected]>
MaureenHelm
pushed a commit
that referenced
this pull request
Aug 2, 2024
Remove the duplicate `USE_STM32_HAL_RAMCFG` Kconfig symbol definition from Kconfig.stm32, which was introduced by accident in PR #66181, commit 6ed002d. Signed-off-by: Mathieu Choplain <[email protected]>
Chenhongren
pushed a commit
to Chenhongren/zephyr
that referenced
this pull request
Aug 29, 2024
Remove the duplicate `USE_STM32_HAL_RAMCFG` Kconfig symbol definition from Kconfig.stm32, which was introduced by accident in PR zephyrproject-rtos#66181, commit 6ed002d. (cherry picked from commit d50e190) Original-Signed-off-by: Mathieu Choplain <[email protected]> GitOrigin-RevId: d50e190 Cr-Build-Id: 8740450565536499009 Cr-Build-Url: https://cr-buildbucket.appspot.com/build/8740450565536499009 Change-Id: Iccf114a72617301577d808f96a753adce992480f Reviewed-on: https://chromium-review.googlesource.com/c/chromiumos/third_party/zephyr/+/5761295 Tested-by: ChromeOS Prod (Robot) <[email protected]> Commit-Queue: Jeremy Bettis <[email protected]> Reviewed-by: Jeremy Bettis <[email protected]> Tested-by: Jeremy Bettis <[email protected]>
mariucker
pushed a commit
to mariucker/zephyr
that referenced
this pull request
Dec 12, 2024
Remove the duplicate `USE_STM32_HAL_RAMCFG` Kconfig symbol definition from Kconfig.stm32, which was introduced by accident in PR zephyrproject-rtos#66181, commit 6ed002d. Signed-off-by: Mathieu Choplain <[email protected]>
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
Add BLE support on STM32WBA:
Prior building a Zephyr BLE application, user should run 'west blobs fetch stm32' to install binary blobs providing the BLE controller support.
Current limitations, which will be fixed in subsequent PRs:
For now BLE is only compatible with
nucleo_wba55cgboard.Since binary blob available in https://github.com/STMicroelectronics/STM32CubeWBA are not compatible with
nucleo_wba52cgboard, Zephyr BLE application won't build on this target.Based on #65467, as it is based on STM32WBA V1.2.0 package.Inlcudes 7bad12e to avoid CI related build issues.